Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Batch Changes #956

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Batch Changes #956

wants to merge 12 commits into from
+987 −13

Conversation

fenndragon
Copy link

@fenndragon fenndragon commented Sep 23, 2024 •
edited
Loading

Description

This is a batch of changes I made for Floof station being ported over to the upstream branch

Changelog

🆑

  • add: Added new neutral xenos, events, and reagent slimes

@fenndragon
batch changes
913c84c 
batch of changes I made for Floof station ported over to the upstream branch
@github-actions github-actions bot added Changes: Sprite Changes any png or json in an RSI Changes: YML Changes any yml files labels Sep 23, 2024
@fenndragon fenndragon marked this pull request as draft September 23, 2024 20:30
@SimpleStation14 SimpleStation14 changed the title batch changes Batch Changes Sep 23, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 23, 2024 •
edited
Loading

RSI Diff Bot; head commit d714f5e merging into 08248ff
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Mobs/Aliens/Xenos/rouny.rsi

State Old New Status
crit Modified
dead Modified
running Modified
sleeping Modified

Resources/Textures/Mobs/Animals/patriarch.rsi

State Old New Status
patriarch Added
patriarch_crit Added
patriarch_dead Added
patriarch_sleeping Added

Resources/Textures/Mobs/Animals/subject7355.rsi

State Old New Status
glow Added
subject7355 Added
subject7355_crit Added
subject7355_dead Added
subject7355_sleeping Added

Edit: diff updated after d714f5e

SleepyScarecrow
SleepyScarecrow suggested changes Sep 23, 2024
View reviewed changes
Copy link
Contributor

@SleepyScarecrow SleepyScarecrow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just nitpicky directory stuff from me :3

Resources/Prototypes/Floof/Body/Organs/Friendstomach.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Floof/Body/Prototypes/Friendshaped.yml Outdated Show resolved Hide resolved
@fenndragon
folder moved
b0205c0 
folders moved appropriately
@fenndragon
fix
3824ed6 
fixed file structure
@fenndragon fenndragon marked this pull request as ready for review September 23, 2024 21:46
@github-actions github-actions bot added the Status: Needs Review Someone please review this label Sep 23, 2024
Mnemotechnician
Mnemotechnician suggested changes Sep 23, 2024
View reviewed changes
Copy link
Contributor

@Mnemotechnician Mnemotechnician left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The entire changelog should be just 1-2 entries, and should not include technical details.

@fenndragon
Copy link
Author

The entire changelog should be just 1-2 entries, and should not include technical details.

better?

@Mnemotechnician
Copy link
Contributor

It should also be descriptive for someone who's reading it in the menu. "Added new neutral xeno prototypes for admins to use in events" or something similar would be good.

@fenndragon
Copy link
Author

It should also be descriptive for someone who's reading it in the menu. "Added new neutral xeno prototypes for admins to use in events" or something similar would be good.

I listed a bit better what all was added but I did it in one entry, better?

Mnemotechnician
Mnemotechnician approved these changes Sep 24, 2024
View reviewed changes
Copy link
Contributor

@Mnemotechnician Mnemotechnician left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yea, that's fine now.

FoxxoTrystan
FoxxoTrystan reviewed Sep 24, 2024
View reviewed changes
Copy link
Member

@FoxxoTrystan FoxxoTrystan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Except for Rouney Sprite, i dont think those additions are needed.

Those Neutral things were things you done mostly for floof, il let solidus view it.

@FoxxoTrystan FoxxoTrystan added Priority: 4-Low Should be resolved at some point Size: 4-Small For small issues/PRs Type: Port Brings something to here from another codebase labels Sep 24, 2024
@fenndragon
Copy link
Author

Except for Rouney Sprite, i dont think those additions are needed.

Those Neutral things were things you done mostly for floof, il let solidus view it.

it was requested.

@fenndragon
Copy link
Author

Except for Rouney Sprite, i dont think those additions are needed.

Those Neutral things were things you done mostly for floof, il let solidus view it.

Also, how is additional events and reagent slimes not needed? this is literally just additional content.

VMSolidus
VMSolidus approved these changes Sep 24, 2024
View reviewed changes
Copy link
Member

@VMSolidus VMSolidus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wuh>? This is fine though?

@VMSolidus
Copy link
Member

Or well, it would be when the Linter fail is fixed. That's legitimate.

@fenndragon
Copy link
Author

Or well, it would be when the Linter fail is fixed. That's legitimate.

whoops forgot to change the texture paths, my bad

@github-actions github-actions bot added the Changes: Localization Changes any ftl files label Sep 25, 2024
@fenndragon
Update xenopet.yml
6d9251b 
added 3 more spawn types for vent critters 

Signed-off-by: fenndragon <[email protected]>
@fenndragon
Update vent-critters.ftl
8d1a284 
added all event text 

Signed-off-by: fenndragon <[email protected]>
@fenndragon
Copy link
Author

went ahead and just added 3 more events and fixed the event text in the locale while I was thinking about it

@fenndragon
Update xeno.yml
08287ac 
Signed-off-by: fenndragon <[email protected]>
@fenndragon
Update xenopet.yml
d714f5e 
Signed-off-by: fenndragon <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VMSolidus VMSolidus VMSolidus approved these changes

@Mnemotechnician Mnemotechnician Mnemotechnician approved these changes

@SleepyScarecrow SleepyScarecrow SleepyScarecrow approved these changes

@Peptide90 Peptide90 Awaiting requested review from Peptide90

@Pspritechologist Pspritechologist Awaiting requested review from Pspritechologist

@FoxxoTrystan FoxxoTrystan Awaiting requested review from FoxxoTrystan

At least 2 approving reviews are required to merge this pull request.

Assignees

@fenndragon fenndragon

Labels
Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an RSI Changes: YML Changes any yml files Priority: 4-Low Should be resolved at some point Size: 4-Small For small issues/PRs Status: Needs Review Someone please review this Type: Port Brings something to here from another codebase
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@fenndragon @Mnemotechnician @VMSolidus @FoxxoTrystan @SleepyScarecrow